Conversation
|
🍅 Не пройден линтинг или базовые тесты |
|
🍅 Не пройден линтинг или базовые тесты |
|
🍏 Пройдено тестов 15 из 15 |
lego.js
Outdated
There was a problem hiding this comment.
Прочитай гайды по оформлению, пожалуйста. Нельзя называть переменные транслитом
lego.js
Outdated
There was a problem hiding this comment.
array -- плохое название переменной
оно не отражает сути
lego.js
Outdated
There was a problem hiding this comment.
Ты тут возвращаешь значение, названия таких функций принято начинать со слова get
lego.js
Outdated
lego.js
Outdated
There was a problem hiding this comment.
copy? Что можно понять по переменной, которая называется копия?
lego.js
Outdated
There was a problem hiding this comment.
Новая коллекция? Почему она новая? Чем отличается от старой?)
lego.js
Outdated
There was a problem hiding this comment.
С copy та же самая история, что и выше
|
🍅 |
|
🍏 Пройдено тестов 15 из 15 |
|
🍏 |
|
🚀 |
lego.js
Outdated
There was a problem hiding this comment.
Object.assign -- это ES2015. Мы пока остаёмся в рамках ES5, так что эту функцию использовать нельзя
| }) | ||
| .forEach (function (query) { | ||
| copyCollection = query(copyCollection); | ||
| }); |
There was a problem hiding this comment.
В качестве упражнения можете этот кусочек переделать на reduce, тогда можно будет переменную copyCollection не изменять. Но тут я не настаиваю.
lego.js
Outdated
There was a problem hiding this comment.
sort ожидает, что компаратор возвращает число. А у вас возвращается логическое значение. И все значения, для которых условие не выполняются, преобразуются в 0, означающее, что значения равны. Так что функция сортировки корректно у вас работать не будет.
lego.js
Outdated
|
🍅 Не пройден линтинг или базовые тесты |
|
🍅 Не пройден линтинг или базовые тесты |
|
🍅 Не пройден линтинг или базовые тесты |
|
🍅 Не пройден линтинг или базовые тесты |
|
🍅 Не пройден линтинг или базовые тесты |
|
🍅 Пройдено тестов 12 из 15 |
|
🍅 Пройдено тестов 12 из 15 |
|
🍅 Пройдено тестов 11 из 15 |
No description provided.